Skip to content

ENT-14214: Updated linter for event driven cfengine#166

Merged
olehermanse merged 1 commit into
cfengine:mainfrom
victormlg:updated-linter
Jul 15, 2026
Merged

ENT-14214: Updated linter for event driven cfengine#166
olehermanse merged 1 commit into
cfengine:mainfrom
victormlg:updated-linter

Conversation

@victormlg

@victormlg victormlg commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Depends on: cfengine/core#6202

Signed-off-by: Victor Moene <victor.moene@northern.tech>
@cf-bottom

Copy link
Copy Markdown

Thanks for submitting a pull request! Maybe @craigcomstock can review this?

@olehermanse olehermanse added the after-release Waiting for a release to happen before merging. label Jul 1, 2026
],
"status": "normal"
},
"watch": {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"watch": {
"reactor": {

bundle notavalidtype my_bundle
^-----------^
Error: Bundle type must be one of (agent, common, edit_line, edit_xml, monitor, server), not 'notavalidtype' at tests/lint/004_block_type.x.cf:1:8
Error: Bundle type must be one of (agent, common, edit_line, edit_xml, monitor, server, watch), not 'notavalidtype' at tests/lint/004_block_type.x.cf:1:8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Error: Bundle type must be one of (agent, common, edit_line, edit_xml, monitor, server, watch), not 'notavalidtype' at tests/lint/004_block_type.x.cf:1:8
Error: Bundle type must be one of (agent, common, edit_line, edit_xml, monitor, server, reactor), not 'notavalidtype' at tests/lint/004_block_type.x.cf:1:8

@olehermanse olehermanse removed the after-release Waiting for a release to happen before merging. label Jul 15, 2026
@olehermanse olehermanse merged commit ef5f6e5 into cfengine:main Jul 15, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants